Change package_method promise behavior to only allow installing packages which are available#5978
Change package_method promise behavior to only allow installing packages which are available#5978craigcomstock wants to merge 1 commit intocfengine:masterfrom
Conversation
…ges which are available Ticket: ENT-13536 Changelog: title
|
|
||
| default: | ||
| if (!matches) // why do we schedule a 'not matched' operation? | ||
| if (!matches) |
There was a problem hiding this comment.
What if the package is a meta package? "yum install snmpd" would install net-snmpd so in that case this change would block an "snmpd" package_method promiser and I think that is OK. If we really wanted to we could amend the package_list_command to include the meta package names?
There was a problem hiding this comment.
The problem here might be that we only don't want to schedule the operation if it is an Add. If the operation is a Verify we want to let it through to SchedulePackageOp().
|
46 acceptance test failures related to packages. I am not surprised... I am not sure what the mock package manager returns for a list of installed packages... maybe NOTHING? So that would make sense that a lot of things would start failing :) |
|
put this on hold for now... need to work on fixing things up so they really "work" :) |
1 participant
Ticket: ENT-13536
Changelog: title